Skip to content

Conversation

@gabrielmfern
Copy link
Member

@gabrielmfern gabrielmfern commented Oct 2, 2025


Summary by cubic

Migrated the Raycast magic link email to Tailwind v4 for React Email, replacing inline styles with utility classes for easier maintenance and consistent styling. Addresses Linear OSS-233.

  • Refactors
    • Wrapped the template with Tailwind and imported the shared config.
    • Converted all inline styles to Tailwind classes (layout, text, colors, hr, grayscale).
    • Added a custom raycast font family in tailwind.config.ts and applied it via font-raycast.

@gabrielmfern gabrielmfern self-assigned this Oct 2, 2025
@changeset-bot
Copy link

changeset-bot bot commented Oct 2, 2025

⚠️ No Changeset found

Latest commit: 9b36819

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Oct 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
react-email Canceled Canceled Oct 10, 2025 7:36pm
react-email-demo Canceled Canceled Oct 10, 2025 7:36pm

@alwaysmeticulous
Copy link

🤖 Meticulous was unable to start a test run because no sessions have been recorded yet. Click here to setup session recording.

Last updated for commit 9da6877. This comment will update as new commits are pushed.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@gabrielmfern gabrielmfern marked this pull request as draft October 2, 2025 15:04
@gabrielmfern gabrielmfern changed the title raycast-magic-link.tsx feat(demo): use Tailwind v4 in raycast-magic-link.tsx Oct 2, 2025
@gabrielmfern gabrielmfern marked this pull request as ready for review October 2, 2025 20:27
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 2 files

Prompt for AI agents (all 1 issues)

Understand the root cause of the following 1 issues and fix them.


<file name="apps/demo/emails/magic-links/raycast-magic-link.tsx">

<violation number="1" location="apps/demo/emails/magic-links/raycast-magic-link.tsx:44">
The new tailwind class leading-6.5 never gets generated: Tailwind’s default line-height scale (and this config) only exposes integer keys like 6 or 7, so the browser drops the class and the text falls back to the component’s default line-height instead of the intended 26px. Swap this for an arbitrary value (e.g. leading-[26px]) to match the original spacing.</violation>
</file>

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

@gabrielmfern gabrielmfern force-pushed the feature/oss-233-raycast-magic-linktsx branch from b4e6189 to 67a3d45 Compare October 10, 2025 14:04
@gabrielmfern gabrielmfern merged commit 1d79af1 into chore/basic-setup Oct 10, 2025
8 of 10 checks passed
@gabrielmfern gabrielmfern deleted the feature/oss-233-raycast-magic-linktsx branch October 10, 2025 19:29
@vercel vercel bot temporarily deployed to Preview – react-email October 10, 2025 19:33 Inactive
@vercel vercel bot temporarily deployed to Preview – react-email-demo October 10, 2025 19:36 Inactive
gabrielmfern added a commit that referenced this pull request Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants